Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci(.github/workflows/lint.yml): only install necessary tools #4

Merged
merged 39 commits into from
Oct 14, 2024
Merged

ci(.github/workflows/lint.yml): only install necessary tools #4

merged 39 commits into from
Oct 14, 2024

Conversation

risu729
Copy link
Owner

@risu729 risu729 commented Jun 3, 2024

No description provided.

@risu729 risu729 linked an issue Jun 3, 2024 that may be closed by this pull request
Only install required tools in CI #2
Closed
@risu729
Copy link
Owner Author

risu729 commented Jun 3, 2024

mise somehow crashes lol

@risu729 risu729 enabled auto-merge (squash) July 15, 2024 01:43
@risu729 risu729 disabled auto-merge September 23, 2024 11:36
@risu729 risu729 force-pushed the 2-only-install-required-tools-in-ci branch 6 times, most recently from df58ffe to f8c3e4f Compare September 23, 2024 14:07
@risu729 risu729 force-pushed the 2-only-install-required-tools-in-ci branch from f8c3e4f to 5711893 Compare September 23, 2024 14:08
@risu729
Revert "ci(.github/workflows/lint.yml): show content of msgpack mise …
f29b335 
…cache"

This reverts commit 5711893.
@risu729
ci(.github/workflows/lint.yml): list all mise cache files for debug
8b6fc13
@risu729
Revert "ci(.github/workflows/lint.yml): list all mise cache files for…
806ab7d 
… debug"

This reverts commit 8b6fc13.
@risu729
ci(.github/workflows/lint.yml): disable mise auto install
de27ea9
@risu729
ci(.github/workflows/lint.yml): exchange the job disabling mise auto …
1483b9f 
…instlal
@risu729
ci(.github/workflows/lint.yml): enable logging
4d91af5
@risu729
ci(.github/workflows/lint.yml): limit rayon max number of threads
388880b
@risu729
ci(.github/workflows/lint.yml): remove lines for debug
a892e67
@risu729
ci(.github/workflows/lint.yml): remove node from mise shims
c4a7878
@risu729
Revert "ci(.github/workflows/lint.yml): remove node from mise shims"
d05cf77 
This reverts commit c4a7878.
@risu729
ci(.github/workflows/lint.yml): run mise for debug
d60cfc7
risu729 and others added 14 commits October 4, 2024 06:22
@risu729
Merge branch 'main' into 2-only-install-required-tools-in-ci
e3bcd5a
@risu729
ci(.github/workflows/lint.yml): cleanup lines for debug
1aca8d1
@risu729
ci(.github/workflows/lint.yml): use install_args for mise install
870ec9a
@risu729
Revert "fix(.github/workflows/lint.yml): remove second mise install a…
eaaefff 
…ttempt"

This reverts commit 086a6ad.
@risu729
ci(.github/workflows/lint.yml): remove mise reshim for non-background…
b29816c 
… related cases
@risu729
ci(.github/workflows/lint.yml): remove npm shim to avoid mise infinit…
409b712 
…e loop
@risu729
ci(.github/workflows/lint.yml): output which npm for debug
df97378
@risu729
ci(.github/workflows/lint.yml): debug
31979ac
@risu729
ci(.github/workflows/lint.yml): remove whole shims directory for debug
e8f2ee8
@risu729
Revert "ci(.github/workflows/lint.yml): remove whole shims directory …
16d3d2d 
…for debug"

This reverts commit e8f2ee8.
@risu729
Revert "ci(.github/workflows/lint.yml): debug"
8a2cd37 
This reverts commit 31979ac.
@risu729
Revert "ci(.github/workflows/lint.yml): output which npm for debug"
f97fc40 
This reverts commit df97378.
@risu729
ci(.github/workflows/lint.yml): set mise log level to trace
1ea68c8
@risu729
ci(.github/workflows/lint.yml): show npm binary for debug
b53ec74
@risu729
Copy link
Owner Author

risu729 commented Oct 4, 2024 •
edited
Loading

I need to exclude node from mise completely to avoid mise infinite loop as npm calls node inside.
jdx/mise#2682

@risu729 risu729 enabled auto-merge (squash) October 14, 2024 11:59
@risu729 risu729 merged commit 228cf9c into main Oct 14, 2024
26 checks passed
@risu729 risu729 deleted the 2-only-install-required-tools-in-ci branch October 14, 2024 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Only install required tools in CI
1 participant
@risu729